Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update AutoGluon to 1.0 API #604

Merged
merged 2 commits into from
Jan 30, 2024
Merged

Update AutoGluon to 1.0 API #604

merged 2 commits into from
Jan 30, 2024

Conversation

Innixma
Copy link
Collaborator

@Innixma Innixma commented Jan 8, 2024

Made a few updates to align AMLB with AutoGluon 1.0's API.

Note that all changes are backwards compatible. For AutoGluon 1.0 exclusive logic, I added if/else checks so it continues to work with older versions: Further, the mainline AMLB code works with AutoGluon 1.0 already as old methods were deprecated but not removed, it will just log deprecation warnings.

Changes related to AutoGluon 1.0:

  1. persist_models -> persist
  2. persist max_memory=0.4 -> removed as it is now the default
  3. predictor.predict_from_proba to allow for decision threshold calibration (not enabled by default)
  4. Remove silent=True to leaderboard call, as it is now silent by default.

General improvements unrelated to AutoGluon 1.0:

  1. Allowed config.max_runtime_seconds to be None when good_quality or high_quality preset is specified and avoid crashing.
  2. set -e: Ensure install script crashes when an error occurs rather than continuing (previously could lead to corrupted installs sneaking by and being ran on datasets, now they should error properly)
  3. Switch repo location from awslabs/autogluon to autogluon/autogluon (it was migrated last year, awslabs/autogluon redirects to autogluon/autogluon)

@Innixma Innixma requested a review from PGijsbers January 8, 2024 22:53
Copy link
Collaborator

@PGijsbers PGijsbers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay

@PGijsbers PGijsbers merged commit fe33eab into openml:master Jan 30, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants